This repository has been archived by the owner on Nov 1, 2024. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 59
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Update comments that set off a false positive in the `unintended_html_in_doc_comment` lint. For references to core Dart types with generics, replace the square braces with backticks since it is useful to constantly link them. For the comments that are edited, and a few nearby, update for style: - Separate out a header as it's own paragraph. - Reduce repetitive phrasing and redundant links to the same arguments. Use a trailing comma for a long argument list, and re-order the arguments so that the always-useful arguments are at the top, and the arguments only pertaining to certain uses follow.
This was referenced Aug 29, 2024
Closed
kevmoo
approved these changes
Aug 29, 2024
copybara-service bot
pushed a commit
to dart-lang/sdk
that referenced
this pull request
Sep 13, 2024
…l, http, lints, mockito, shelf, test, test_descriptor, typed_data, web, webkit_inspection_protocol, yaml Revisions updated by `dart tools/rev_sdk_deps.dart`. args (https://github.com/dart-lang/args/compare/1a24d61..e623652): e623652 2024-08-26 Kevin Moore Fix angle brackets in doc comments (dart-archive/args#281) browser_launcher (https://github.com/dart-lang/browser_launcher/compare/fa98c77..0acd188): 0acd188 2024-08-22 Srujan Gaddam Add CHANGELOG entry for dart-archive/browser_launcher#63 and publish 1.2.2 (dart-archive/browser_launcher#64) collection (https://github.com/dart-lang/collection/compare/0c1f829..24b75d8): 24b75d8 2024-08-26 Kevin Moore Fix angle brackets in doc comments (dart-archive/collection#358) csslib (https://github.com/dart-lang/csslib/compare/192d720..d486627): d486627 2024-08-26 Kevin Moore Fix angle brackets in doc comments (dart-archive/csslib#206) ecosystem (https://github.com/dart-lang/ecosystem/compare/8626bff..4ba6f15): 4ba6f15 2024-09-04 Devon Carew update the prompt to include `needs-info` (dart-lang/ecosystem#296) 3496f54 2024-09-02 Moritz Continue with labeling after error with issue transfer (dart-lang/ecosystem#293) bb45d73 2024-09-02 Moritz Run `dart pub global run mono_repo generate` (dart-lang/ecosystem#295) 8d5860b 2024-09-01 dependabot[bot] Bump the github-actions group with 3 updates (dart-lang/ecosystem#297) 6de3e8c 2024-08-29 Devon Carew fix calculation for the labels to apply (dart-lang/ecosystem#294) 183fdd0 2024-08-28 Devon Carew contribute a prompt benchmarking script (dart-lang/ecosystem#292) f7191b7 2024-08-28 Moritz Transfer issues in `package:trebuchet` (dart-lang/ecosystem#291) 5da8a7a 2024-08-26 Moritz Add `package:trebuchet` to repo. (dart-lang/ecosystem#289) dd8d12a 2024-08-23 Nate Bosch Add check label to github action job name (dart-lang/ecosystem#290) html (https://github.com/dart-lang/html/compare/0da420c..5516387): 5516387 2024-08-28 Kevin Moore Drop Pair util class (dart-archive/html#255) 50837e5 2024-08-28 Nate Bosch Fix doc comments (dart-archive/html#256) http (https://github.com/dart-lang/http/compare/b97b8dc..dfeecf0): dfeecf0 2024-08-28 Brian Quinlan Convert a `java.lang.OutOfMemoryError` into a `ClientException` (dart-lang/http#1299) a41141d 2024-08-28 Brian Quinlan cupertino_http: Use `sharedDarwinSource` (dart-lang/http#1290) 51beb4e 2024-08-28 Yaroslav Vorobev fix(http_client_conformance_tests): multipart server (dart-lang/http#1292) bbfce40 2024-08-28 Brian Quinlan Add a reference to AdapterWebSocketChannel for WebSocketChannel users (dart-lang/http#1297) 7f21111 2024-08-21 Moritz Add PR Health workflow (dart-lang/http#1296) lints (https://github.com/dart-lang/lints/compare/894b500..9e4fd7d): 9e4fd7d 2024-09-10 Lenz Paul Fixes formatting error in the rules.md table (dart-lang/lints#206) 6580f83 2024-09-05 Sam Rawlins Remove avoid_null_checks_in_equality_operators from recommended (dart-lang/lints#201) mockito (https://github.com/dart-lang/mockito/compare/eb4d1da..d0fda0c): d0fda0c 2024-09-11 Sam Rawlins Bump CI tasks to use Dart 3.5.0 as stable 2259e10 2024-09-11 Sam Rawlins Pass a Dart language version to the Dart formatter. d7a8334 2024-09-10 Sam Rawlins mockito: stop using deprecated analyzer APIs. shelf (https://github.com/dart-lang/shelf/compare/9f2dffe..d53a8f9): d53a8f9 2024-09-06 Nate Bosch Prepare to publish shelf_static (dart-lang/shelf#443) e725c73 2024-09-06 mx1up Loosen mime package constraint (dart-lang/shelf#441) efbc566 2024-09-05 Nate Bosch Remove links to core types with generics (dart-lang/shelf#442) test (https://github.com/dart-lang/test/compare/cd3dbd5..9a2d155): 9a2d155b 2024-09-11 Nate Bosch Run package:test tests on mac (dart-lang/test#2280) 569c6eb6 2024-09-01 dependabot[bot] Bump the github-actions group with 2 updates (dart-lang/test#2275) db8cf091 2024-08-28 Nate Bosch Remove the Pair utility (dart-lang/test#2271) 032ef1dc 2024-08-27 Simon Binder Support dart2wasm in node.js tests (dart-lang/test#2259) 90481cf8 2024-08-23 Nate Bosch Limit which health checks run (dart-lang/test#2270) 337c7fb0 2024-08-21 Moritz Update publish.yaml (dart-lang/test#2269) test_descriptor (https://github.com/dart-lang/test_descriptor/compare/90743bc..3b85d38): 3b85d38 2024-08-26 Kevin Moore Fix angle brackets in doc comments (dart-lang/test_descriptor#70) typed_data (https://github.com/dart-lang/typed_data/compare/365468a..2bb9e6e): 2bb9e6e 2024-08-26 Kevin Moore Require Dart 3.5, use TypeDataList in many places (dart-archive/typed_data#92) web (https://github.com/dart-lang/web/compare/fb30192..933a37d): 933a37d 2024-09-10 Srujan Gaddam Update dart_style and use latestLanguageVersion (dart-lang/web#301) webkit_inspection_protocol (https://github.com/google/webkit_inspection_protocol.dart/compare/119b877..e7418d7): e7418d7 2024-08-26 Kevin Moore Update min SDK and a number of lints (google/webkit_inspection_protocol.dart#128) yaml (https://github.com/dart-lang/yaml/compare/a645c39..b3d299e): b3d299e 2024-08-28 Kevin Moore Drop Pair util class (dart-lang/yaml#169) Change-Id: Ic3461659a40427c0deaafab8c79cade332f76edb Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/385302 Reviewed-by: Konstantin Shcheglov <[email protected]> Auto-Submit: Devon Carew <[email protected]> Commit-Queue: Konstantin Shcheglov <[email protected]>
mosuem
pushed a commit
to dart-lang/tools
that referenced
this pull request
Oct 25, 2024
Update comments that set off a false positive in the `unintended_html_in_doc_comment` lint. For references to core Dart types with generics, replace the square braces with backticks since it isn't useful to constantly link them. For the comments that are edited, and a few nearby, update for style: - Separate out a header as it's own paragraph. - Reduce repetitive phrasing and redundant links to the same arguments. Use a trailing comma for a long argument list, and re-order the arguments so that the always-useful arguments are at the top, and the arguments only pertaining to certain uses follow.
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Update comments that set off a false positive in the
unintended_html_in_doc_comment
lint. For references to core Darttypes with generics, replace the square braces with backticks since it
isn't useful to constantly link them.
For the comments that are edited, and a few nearby, update for style:
Use a trailing comma for a long argument list, and re-order the
arguments so that the always-useful arguments are at the top, and the
arguments only pertaining to certain uses follow.